-
Notifications
You must be signed in to change notification settings - Fork 116
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update readme
for w.org plugins page to match new copy
#1114
Conversation
readme.txt
Outdated
@@ -1,24 +1,26 @@ | |||
=== Stream === | |||
Contributors: lukecarbis, fjarrett, stream, xwp, kasparsd | |||
Contributors: xwp, kasparsd, derekherman, postphotos |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably just be xwp
. Not sure who has access to the stream account on wp.org and this list gives everyone write access to SVN. @kasparsd I can either give you the credentials to the xwp
account so you can still push new versions of the plugin or keep you on the list but I don't see why I or Leo would be on this list. Thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. Just pushed a change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure who has access to the stream account on wp.org and this list gives everyone write access to SVN
@derekherman This contributors list is only included in the plugin credits and doesn't give SVN access.
This attribution also makes the plugin show up on user's list of contributions which is nice:
The SVN access is managed via Committers under the plugin settings:
Hey @derekherman - I've added the list of contributors as @kasparsd suggested and removed the other non-repo-committing folks from the list. I think this is good to merge now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good!
We can iterate on the contributor credits as needed so I'm merging this in.
Updated readme to match new copy that more accurately describes Stream.